Skip to content

Task:4019 Upgrade type hints to 3.10+ style #4113

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

Task:4019 Upgrade type hints to 3.10+ style #4113

wants to merge 2 commits into from

Conversation

@Json-Andriopoulos
Copy link
Contributor

@Json-Andriopoulos Json-Andriopoulos commented Oct 30, 2025

Describe changes

I implemented/fixed _ to achieve _.

Pre-requisites

Please ensure you have done the following:

  • I have read the CONTRIBUTING.md document.
  • I have added tests to cover my changes.
  • I have based my new branch on develop and the open PR is targeting develop. If your branch wasn't based on develop read Contribution guide on rebasing branch to develop.
  • IMPORTANT: I made sure that my changes are reflected properly in the following resources:
    • ZenML Docs
    • Dashboard: Needs to be communicated to the frontend team.
    • Templates: Might need adjustments (that are not reflected in the template tests) in case of non-breaking changes and deprecations.
    • Projects: Depending on the version dependencies, different projects might get affected.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Other (add details above)

@github-actions
Copy link
Contributor

github-actions bot commented Oct 30, 2025 •
edited
Loading

Documentation Link Check Results

❌ Absolute links check failed
There are broken absolute links in the documentation. See workflow logs for details
✅ Relative links check passed
Last checked: 2025-10-30 15:28:39 UTC

schustmi
schustmi reviewed Oct 31, 2025
View reviewed changes
scripts/pyupgrade.sh

# run py-upgrade recursively to all nested .py files

find src -name '*.py' -print0 | xargs -0 -n1 python -m pyupgrade --py310-plus
Copy link
Contributor

@schustmi schustmi Oct 31, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we can also run it on the tests and examples directories (or any other python files in the repo)?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@schustmi schustmi schustmi left review comments

@stefannica stefannica Awaiting requested review from stefannica

@bcdurak bcdurak Awaiting requested review from bcdurak

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

run-slow-ci

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Json-Andriopoulos @schustmi